Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Multiple choice answer matching anomaly #412

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

n-WN
Copy link
Contributor

@n-WN n-WN commented Dec 26, 2024

core

https://github.com/n-WN/chaoxing_fork/blob/1df9e714e348a252ac1345338f8457f0783c0566/api/base.py#L298

pretreatment

  • Prevents incorrect answers due to poor matches with the question bank.
  • It would be better to maintain the return in terms of correctness, starting with this.

feat

Introduce chapter_test parameter and improve code readability #410

n-WN added 10 commits December 23, 2024 23:33
…tion (Samueli924#405)

- Added help and verbose options for better user control and logging.
- Introduced random sleep between chapters for more realistic simulation.
- Fixed multi_cut function to handle multiple-choice questions correctly.

Fixes: Samueli924#405
- Added chapter_test parameter for configuring chapter quiz handling options:
  - 0: Skip
  - 1: Complete and save
  - 2: Complete and submit
- Implemented functionality to skip quiz tasks when chapter_test is set to 0
- Background: Many users do not have a tiku token configured and do not want to use random selection. This is a simple solution to address that.

Excerpt from main.py, lines 255 to 259:
elif job['type'] == 'workid':
    # 检测配置文件是否跳过测验任务
    if chapter_test == 0:
        logger.info(f'跳过章节测验任务, 任务章节: {course['title']}')
        continue
- Optimize course list parsing from config files and command line arguments
- Add parse_course_list function to handle course IDs containing whitespace
- Simplify code with list comprehension

Background:
Previous version used simple .split(,) which ignored course IDs containing
whitespace in configuration. The new parsing mechanism properly handles
whitespace in course ID lists.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant